Skip to content

Update ag-neuro links and formatting #1766

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Sep 2, 2025
Merged

Update ag-neuro links and formatting #1766

merged 2 commits into from
Sep 2, 2025

Conversation

rly
Copy link
Contributor

@rly rly commented Aug 28, 2025 •
edited
Loading

Description

  • Fixed broken cvnet mailing list link
  • Added NeuroTechX job board link
  • Updated social media list
  • Fixed link formatting

Checklist:

  • I have previewed changes locally or with CircleCI (runs when PR is created)
  • I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

@rly
Copy link
Contributor Author

rly commented Aug 28, 2025

Requesting review from @billbrod and/or @NickleDave

@rly
Copy link
Contributor Author

rly commented Aug 28, 2025

Failing links are unrelated to this PR.

billbrod
billbrod approved these changes Sep 2, 2025
View reviewed changes
Copy link
Contributor

@billbrod billbrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

I'm unable to find (/ have forgotten how to find) the preview build of the website, so I'm just looking at the diff in the PR.

@mrmundt
Copy link
Contributor

mrmundt commented Sep 2, 2025

Looks good to me!

I'm unable to find (/ have forgotten how to find) the preview build of the website, so I'm just looking at the diff in the PR.

Ask and ye shall receive: https://output.circle-artifacts.com/output/job/c32fca94-b322-4a0d-907f-612a51cd26e7/artifacts/0/usrse.github.io/ag/neuro-rse/index.html

@billbrod
Copy link
Contributor

billbrod commented Sep 2, 2025

Looks good to me!
I'm unable to find (/ have forgotten how to find) the preview build of the website, so I'm just looking at the diff in the PR.

Ask and ye shall receive: https://output.circle-artifacts.com/output/job/c32fca94-b322-4a0d-907f-612a51cd26e7/artifacts/0/usrse.github.io/ag/neuro-rse/index.html

Thanks Miranda! I looked at the preview build and confirm this looks good.

@mrmundt
Copy link
Contributor

mrmundt commented Sep 2, 2025

I'm happy to click "Merge" if y'all are good with this PR.

@mrmundt mrmundt merged commit d0000af into USRSE:main Sep 2, 2025
1 of 2 checks passed
@rly rly deleted the patch-1 branch September 2, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@billbrod billbrod billbrod approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rly @mrmundt @billbrod